-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: ephemeral peer channels #59
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good -- so this allows to let the hello example use realtime channels?
It just adds the api - not using it. But since Hello is the entry point for webxdc development I suppose this should be here @hpk42 |
Not sure -- realtime is still experimental and if there is no support
for using it, then it doesn't really help. Otherwise the hello-app
does allow to use sendUpdate/setUpdateListener and that works from multiple tabs, no?
…On Mon, May 13, 2024 at 07:36 -0700, Sebastian Klähn wrote:
It just adds the api - not using it. But since Hello is the entry point for webxdc development I suppose this should be here @hpk42
--
Reply to this email directly or view it on GitHub:
#59 (comment)
You are receiving this because you were mentioned.
Message ID: ***@***.***>
|
Yes that works for multiple tabs. Wanted to cross this off my mental list, but we can also keep it open until realtime API actually arrived on the platforms |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, did not test
Part of 5346